Skip to content

Conformity of client interceptor(s) with the official grpc.*ClientInterceptor-interfaces#3877

Closed
juandelacruz-calvo wants to merge 5 commits into
open-telemetry:mainfrom
juandelacruz-calvo:pr-3384-conformity
Closed

Conformity of client interceptor(s) with the official grpc.*ClientInterceptor-interfaces#3877
juandelacruz-calvo wants to merge 5 commits into
open-telemetry:mainfrom
juandelacruz-calvo:pr-3384-conformity

Conversation

@juandelacruz-calvo

@juandelacruz-calvo juandelacruz-calvo commented Oct 17, 2025

Copy link
Copy Markdown
Contributor

Summary

This PR is a rebased version of PR #3384 that aims to update the OpenTelemetryClientInterceptor to support official gRPC client interceptor interfaces introduced in gRPC v1.8.0.

Changes

  • Remove external implementation of gRPC client interceptors (package grpcext)
  • Adapt the _client.py module based on the _aio_client.py module design
  • Remove the RpcInfo class from utilities.py
  • Update test_client_interceptor_trace_context_propagation tests
  • Update semantic conventions to use the new incubating attributes

Test plan

Tested locally, working with grpc client instrumentation, also working with PubSub consuming and publishing

Related Issues

🤖 Generated with Claude Code

…rceptor-interfaces

Signed-off-by: Jason Parraga <sovietaced@gmail.com>
Signed-off-by: Jason Parraga <sovietaced@gmail.com>
@juandelacruz-calvo juandelacruz-calvo marked this pull request as ready for review October 17, 2025 14:32
@juandelacruz-calvo

juandelacruz-calvo commented Oct 20, 2025

Copy link
Copy Markdown
Contributor Author

This is really no longer needed to fix the issue for PubSub on GCP, since they released specific properties to instrument their software with OpenTelemetry: https://cloud.google.com/pubsub/docs/open-telemetry-tracing

But it helps to add manual intercepts to gRPC clients

@xrmx xrmx moved this to Ready for review in Python PR digest Oct 31, 2025
@juandelacruz-calvo

juandelacruz-calvo commented Nov 14, 2025

Copy link
Copy Markdown
Contributor Author

Apologies for the tag @xrmx

There was an issue with my GH token.

@linux-foundation-easycla

linux-foundation-easycla Bot commented Nov 14, 2025

Copy link
Copy Markdown

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: juandelacruz-calvo / name: Juan Calvo Pozo (2ed4321)

@juandelacruz-calvo

Copy link
Copy Markdown
Contributor Author

Superseded by #4259

@github-project-automation github-project-automation Bot moved this from Ready for review to Done in Python PR digest Feb 25, 2026
@juandelacruz-calvo juandelacruz-calvo deleted the pr-3384-conformity branch February 25, 2026 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants